Skip to content

Conversation

@kodster28
Copy link
Collaborator

Summary

With the work in #22754, we're going to need to be much more careful about the "outdated, but still referenceable content in our docs".

An example of this is Wrangler 1 commands.

We still need these for reference though, so we'll be adding the noindex flag to them as a middle path.

@kodster28 kodster28 requested review from a team and irvinebroque as code owners May 29, 2025 19:13
@github-actions github-actions bot added the product:workers Related to Workers product label May 29, 2025
To deploy this example Worker to the `helloworld` environment, you would run `wrangler publish --env helloworld`.

***
---
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is all just prettier formatting.

@hyperlint-ai
Copy link
Contributor

hyperlint-ai bot commented May 29, 2025

Howdy and thanks for contributing to our repo. The Cloudflare team reviews new, external PRs within two (2) weeks. If it's been two weeks or longer without any movement, please tag the PR Assignees in a comment.

We review internal PRs within 1 week. If it's something urgent or has been sitting without a comment, start a thread in the Developer Docs space internally.


PR Change Summary

Added the noindex flag to Wrangler 1 documentation pages to prevent them from being indexed by search engines while still allowing them to be referenced.

  • Introduced noindex flag to multiple Wrangler 1 documentation pages
  • Ensured outdated content remains accessible for reference
  • Modified several files related to Wrangler 1 commands and configuration

Modified Files

  • src/content/docs/workers/wrangler/migration/v1-to-v2/wrangler-legacy/authentication.mdx
  • src/content/docs/workers/wrangler/migration/v1-to-v2/wrangler-legacy/commands.mdx
  • src/content/docs/workers/wrangler/migration/v1-to-v2/wrangler-legacy/configuration.mdx
  • src/content/docs/workers/wrangler/migration/v1-to-v2/wrangler-legacy/index.mdx
  • src/content/docs/workers/wrangler/migration/v1-to-v2/wrangler-legacy/install-update.mdx
  • src/content/docs/workers/wrangler/migration/v1-to-v2/wrangler-legacy/webpack.mdx

How can I customize these reviews?

Check out the Hyperlint AI Reviewer docs for more information on how to customize the review.

If you just want to ignore it on this PR, you can add the hyperlint-ignore label to the PR. Future changes won't trigger a Hyperlint review.

Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add hyperlint-ignore to the PR to ignore the link check for this PR.

@github-actions
Copy link
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/workers/wrangler/ @cloudflare/wrangler, @irvinebroque, @kodster28, @cloudflare/pcx-technical-writing

@github-actions
Copy link
Contributor

Preview URL: https://4a087d65.preview.developers.cloudflare.com
Preview Branch URL: https://noindex-wrangler-v1.preview.developers.cloudflare.com

Files with changes (up to 15)

Original Link Updated Link
https://developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/configuration/ https://noindex-wrangler-v1.preview.developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/configuration/
https://developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/webpack/ https://noindex-wrangler-v1.preview.developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/webpack/
https://developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/ https://noindex-wrangler-v1.preview.developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/
https://developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/commands/ https://noindex-wrangler-v1.preview.developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/commands/
https://developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/install-update/ https://noindex-wrangler-v1.preview.developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/install-update/
https://developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/authentication/ https://noindex-wrangler-v1.preview.developers.cloudflare.com/workers/wrangler/migration/v1-to-v2/wrangler-legacy/authentication/

Copy link
Contributor

@caley-b caley-b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Live long and prosper to no index-ville.

@kodster28 kodster28 merged commit 8c9b97f into production May 29, 2025
12 checks passed
@kodster28 kodster28 deleted the noindex-wrangler-v1 branch May 29, 2025 19:47
sdnts pushed a commit to sdnts/cloudflare-docs that referenced this pull request Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:workers Related to Workers product size/m

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants